Skip to content

chore(huggingface-hub): Fix chat completion stream type - #7070

Merged
alexander-alderman-webb merged 7 commits into
masterfrom
webb/add-huggingface-hub-types
Aug 7, 2026
Merged

chore(huggingface-hub): Fix chat completion stream type#7070
alexander-alderman-webb merged 7 commits into
masterfrom
webb/add-huggingface-hub-types

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Description

Add huggingface-hub to the uv typing group.
Cast iterator to the correct type.

The type for chat_completion() can be found here:

https://github.com/huggingface/huggingface_hub/blob/b4f4e92226642a4472945a174b9761b688a27dae/src/huggingface_hub/inference/_client.py#L559

Issues

Reminders

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review August 6, 2026 15:07
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner August 6, 2026 15:07
@alexander-alderman-webb
alexander-alderman-webb changed the base branch from renovate/lock-file-maintenance to master August 6, 2026 15:11
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

96974 passed | ⏭️ 6239 skipped | Total: 103213 | Pass Rate: 93.96% | Execution Time: 346m 55s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +11522
Passed Tests 📈 +11368
Failed Tests
Skipped Tests 📈 +154

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2506 uncovered lines.
❌ Project coverage is 89.91%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
- Coverage    89.95%    89.91%    -0.04%
==========================================
  Files          193       193         —
  Lines        24946     24828      -118
  Branches      9000      8930       -70
==========================================
+ Hits         22439     22322      -117
- Misses        2507      2506        -1
- Partials      1435      1416       -19

Generated by Codecov Action

@ericapisani

Copy link
Copy Markdown
Member

Looks like there's some test failures, will wait until they're 🟢 before taking a look

@alexander-alderman-webb
alexander-alderman-webb merged commit 65ec535 into master Aug 7, 2026
381 of 394 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/add-huggingface-hub-types branch August 7, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants